Skip to content

Conversation

@jp-agenta
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings November 12, 2025 09:33
@vercel
Copy link

vercel bot commented Nov 12, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
agenta-documentation Ready Ready Preview Comment Nov 12, 2025 9:34am

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Nov 12, 2025
@jp-agenta jp-agenta changed the title add bump workflow [CI] Add create release branch workflow Nov 12, 2025
@dosubot dosubot bot added the ci/cd label Nov 12, 2025
@junaway junaway merged commit 834c368 into main Nov 12, 2025
14 checks passed
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a GitHub Actions workflow to automate version bumping across multiple project components (web, ee, oss, sdk, api) and create a release branch with a pull request.

  • Automates version bumping using semver for patch, minor, or major releases
  • Creates a release branch and PR when versions match across all components
  • Includes a tag creation job after successful version bump

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- name: Bump versions and compare
id: bump_versions
run: |
BUMP_TYPE=${{ inputs.bump-type }}
Copy link

Copilot AI Nov 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The input parameter is defined as 'release-type' (line 6) but referenced as 'bump-type' here. This should be 'inputs.release-type' to match the workflow input definition.

Suggested change
BUMP_TYPE=${{ inputs.bump-type }}
BUMP_TYPE=${{ inputs.release-type }}

Copilot uses AI. Check for mistakes.

- name: Create Pull Request
if: steps.bump_versions.outputs.VERSIONS_MATCH == 'true'
uses: peter-evans/create-pull-request@v6
Copy link

Copilot AI Nov 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The step is missing an 'id' field. The job output on line 23 references 'steps.create-pr.outputs.pull-request-number', but this step doesn't have 'id: create-pr' defined, which will cause the output reference to fail.

Copilot uses AI. Check for mistakes.
Comment on lines +136 to +141
- name: Create and push tag
run: |
git config --global user.name "${{ github.actor }}"
git config --global user.email "${{ github.actor_id }}+${{ github.actor }}@users.noreply.github.com"
git tag -a "v${{ needs.bump-version.outputs.new_version }}" -m "Version ${{ needs.bump-version.outputs.new_version }}"
git push origin "v${{ needs.bump-version.outputs.new_version }}"
Copy link

Copilot AI Nov 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The create-tag job checks out the repository but doesn't specify which branch or ref to checkout. Since this job runs after the PR is created, it will checkout the default branch (likely main), not the release branch where the version bump was committed. The tag will be created on the wrong commit. Consider adding 'ref: release/v${{ needs.bump-version.outputs.new_version }}' to the checkout step or removing the tag creation until after the PR is merged.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants